linehelper.js ➔ fileReadLinesRange - Code Metrics - Inspection of "Further rewrite" - whois-team/whoisdigger - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 65f9b0...e165e5 )

by Eduardo

created 2020-07-28 21:36 UTC

linehelper.js ➔ fileReadLinesRange A

↳ Parent: app/js/common/linehelper.js

Complexity

Conditions

Size

Total Lines	20
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	14
dl	0
loc	20
rs	9.7
c	0
b	0
f	0
cc	4

// jshint esversion: 8

/*
  lineCount
    Count lines within a string
  parameters
    text (string) - string to count lines from
    newLineChar (string) - new line character
 */
function lineCount(text, newLineChar = '\n') { // '\n' unix; '\r' macos; '\r\n' windows
  var lines = 0;
  for (var char in text) {
var someObject;
for (var key in someObject) {
    if ( ! someObject.hasOwnProperty(key)) {
        continue; // Skip keys from the prototype.
    }

    doSomethingWith(key);
}
    lines += (lines[char] == newLineChar) ? 1 : 0;
  }
  return lines;
}

/*
  fileReadLines
    Read a determined quantity of lines from a specific file
  parameters
    filePath (string) - file path to read lines from
    lines (integer) - line quantity to read from file
    startLine (integer) - line to start reading from
 */
function fileReadLines(filePath, lines = 2, startLine = 0) {
  var lineCounter = startLine,
    endLine = startLine + lines,

    linesRead = [],
    lineReader = require('readline').createInterface({
      input: require('fs').createReadStream(filePath),
    });

  lineReader.on('line', function(line) {
    lineCounter++;
    linesRead.push(line);
    if (lineCounter == lines) {
      lineReader.close();
    }
  });
  lineReader.on('close', function() {
    return linesRead;
  });
}

module.exports = {
  lineCount: lineCount,
  fileReadLines: fileReadLines
};


1			// jshint esversion: 8
2
3			/*
4			lineCount
5			Count lines within a string
6			parameters
7			text (string) - string to count lines from
8			newLineChar (string) - new line character
9			*/
10			function lineCount(text, newLineChar = '\n') { // '\n' unix; '\r' macos; '\r\n' windows
11			var lines = 0;
12			for (var char in text) {
			0 ignored issues – show Complexity introduced 2020-07-28 21:39 UTC by Report Bug Copy Issue Report A for in loop automatically includes the property of any prototype object, consider checking the key using `hasOwnProperty`. When iterating over the keys of an object, this includes not only the keys of the object, but also keys contained in the prototype of that object. It is generally a best practice to check for these keys specifically: var someObject; for (var key in someObject) { if ( ! someObject.hasOwnProperty(key)) { continue; // Skip keys from the prototype. } doSomethingWith(key); } Loading history...
13			lines += (lines[char] == newLineChar) ? 1 : 0;
14			}
15			return lines;
16			}
17
18			/*
19			fileReadLines
20			Read a determined quantity of lines from a specific file
21			parameters
22			filePath (string) - file path to read lines from
23			lines (integer) - line quantity to read from file
24			startLine (integer) - line to start reading from
25			*/
26			function fileReadLines(filePath, lines = 2, startLine = 0) {
27			var lineCounter = startLine,
28			endLine = startLine + lines,
			0 ignored issues – show Unused Code introduced 2020-07-28 21:39 UTC by Report Bug Copy Issue Report The variable `endLine` seems to be never used. Consider removing it. Loading history...
29			linesRead = [],
30			lineReader = require('readline').createInterface({
31			input: require('fs').createReadStream(filePath),
32			});
33
34			lineReader.on('line', function(line) {
35			lineCounter++;
36			linesRead.push(line);
37			if (lineCounter == lines) {
38			lineReader.close();
39			}
40			});
41			lineReader.on('close', function() {
42			return linesRead;
43			});
44			}
45
46			module.exports = {
47			lineCount: lineCount,
48			fileReadLines: fileReadLines
49			};
50

whois-team / whoisdigger

Push — master ( 65f9b0...e165e5 )

linehelper.js ➔ fileReadLinesRange A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like